Skip to content

feat(helm): add custom labels to ServiceMonitor#384

Closed
nikatza wants to merge 1 commit into
mainfrom
expose-prometheus-metrics
Closed

feat(helm): add custom labels to ServiceMonitor#384
nikatza wants to merge 1 commit into
mainfrom
expose-prometheus-metrics

Conversation

@nikatza

@nikatza nikatza commented May 29, 2026

Copy link
Copy Markdown
Contributor

Add prometheus.labels field in values.yaml to allow custom labels on the ServiceMonitor resource. This enables integration with monitoring systems that use labels for scrape discovery.

The Makefile charts target preserves this change when kubebuilder regenerates templates.

Example:

prometheus:
  enable: true
  labels:
    plugin: kube-monitoring

Add prometheus.labels field in values.yaml to allow custom labels
on the ServiceMonitor resource. This enables integration with
monitoring systems that use labels for scrape discovery.

The Makefile charts target preserves this change when kubebuilder
regenerates templates. Adds omit filter to prevent users to overwrite
managed labels.

Signed-off-by: Pujol <enric.pujol@sap.com>
@nikatza nikatza force-pushed the expose-prometheus-metrics branch from c0b281b to c77bb56 Compare June 8, 2026 11:24
@nikatza nikatza marked this pull request as ready for review June 8, 2026 11:24
@nikatza nikatza requested a review from a team as a code owner June 8, 2026 11:24
@nikatza

nikatza commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

@felix-kaestner closing this PR as per the suggestion to use kustomize in our own charts.

@nikatza nikatza closed this Jun 8, 2026
@nikatza nikatza deleted the expose-prometheus-metrics branch June 8, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant